Codecov Report

❗ No coverage uploaded for pull request base (develop@1c5cca4). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff             @@
##             develop     #291   +/-   ##
==========================================
  Coverage           ?   66.56%           
==========================================
  Files              ?      113           
  Lines              ?     7230           
  Branches           ?        0           
==========================================
  Hits               ?     4813           
  Misses             ?     1946           
  Partials           ?      471

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1c5cca4...bd7989a. Read the comment docs.

@AlexStocks Do you think I should capitalize the first letter of methodName for generic invoke because the first letter of go's methodName should be upper.

@AlexStocks Do you think I should capitalize the first letter of methodName for generic invoke because the first letter of go's methodName should be upper.

pls give some adives @fangyincheng @hxmhlt @pantianying

you need a switch to control whether generic_service should be working, instead of default

@fangyincheng In java sdk, generic filter is default filter. In go, do we need to control it be working?

@AlexStocks Do you think I should capitalize the first letter of methodName for generic invoke because the first letter of go's methodName should be upper.

dubbo-go support lowercase method to provide a service.

@fangyincheng In java sdk, generic filter is default filter. In go, do we need to control it be working?

ok

@AlexStocks Do you think I should capitalize the first letter of methodName for generic invoke because the first letter of go's methodName should be upper.

dubbo-go support lowercase method to provide a service.

I see.

Has it been tested with java Dubbo？

@fangyincheng yeah, I tested